-
Notifications
You must be signed in to change notification settings - Fork 0
Split readme docs #114
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Split readme docs #114
Conversation
…x docs with generated md files
…ebooks and config files
.github/workflows/docs.yml
Outdated
| pip install -e .[docs] | ||
| - name: Split README into sections | ||
| run: | | ||
| python docs/split_readme.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will not be done on Sphinx though!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried to preprocess he md files first, and then use them in the Sphinx index, is it not the correct way to do it?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes it's correct. but I wanted to highlight that this has no effect on sphinx. We can have a look at it together tomorrow.
|
This PR closes #79 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is the sphinx-apidoc command here also needed considering it's also done on the sphinx conf.py?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This action just tests that the execution works. As the index.md expects some file under references, it is needed here.
|
Do you want to keep the folder |
enryH
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good.
docs/sections_readme/case_studies.md
Outdated
|
|
||
| VueGen’s functionality is demonstrated through two case studies: | ||
|
|
||
| **1. Predefined Directory** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe you want to add headlines here?
docs/sections_readme/case_studies.md
Outdated
| @@ -0,0 +1,58 @@ | |||
| ## Case studies | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It could be that sphinx will complain about the headlines. Check the logs, but if you split up the README, you could remove one hashtag per headline...
That said, the website looks good, so no big issue
docs/sections_readme/citation.md
Outdated
| @@ -0,0 +1,22 @@ | |||
| ## Citation | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
and it's everywhere in your dumps
… contact to the docs
No description provided.